Skip to content

ci: add code coverage reporting via Codecov and Tarpaulin - #410

Closed
Francis6-git wants to merge 2 commits into
LabsCrypt:mainfrom
Francis6-git:feat/add-codecov-coverage-reporting-433
Closed

ci: add code coverage reporting via Codecov and Tarpaulin#410
Francis6-git wants to merge 2 commits into
LabsCrypt:mainfrom
Francis6-git:feat/add-codecov-coverage-reporting-433

Conversation

@Francis6-git

Copy link
Copy Markdown
Contributor

Description:
This PR addresses Issue #344 by implementing comprehensive code coverage reporting across the stack and preserves the WASM optimization workflow required for Stellar/Soroban deployments.

Key Changes

  • Backend Coverage: Configured Vitest to generate lcov reports with a mandatory 60% coverage threshold.
  • Contract Coverage: Integrated cargo-tarpaulin to generate XML reports for Rust contracts with a mandatory 70% coverage threshold.
  • Codecov Integration: Added GitHub Actions steps to upload coverage data for both backend and contracts using codecov-action@v4.
  • Build Optimizations: Retained the stellar-cli optimization pass to ensure contract WASM artifacts remain within network limits.
  • Visibility: Added a Codecov status badge to the README.md.

CLoses #344

@ogazboiz

Copy link
Copy Markdown
Contributor

hey, main was failing CI from broken auth imports + frontend parse errors. fixed and pushed to main now. please rebase to pick up the fixes:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

if there's a conflict, resolve it locally and we'll review once CI is green.

@Francis6-git

Copy link
Copy Markdown
Contributor Author

hey, main was failing CI from broken auth imports + frontend parse errors. fixed and pushed to main now. please rebase to pick up the fixes:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

if there's a conflict, resolve it locally and we'll review once CI is green.

Conflict resolved

@ogazboiz

ogazboiz commented May 1, 2026

Copy link
Copy Markdown
Contributor

hey, main CI is fully green now. there were several real TypeScript and test issues from #444's merge that have all been fixed. please rebase one more time:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

your PR's diff should be much cleaner after this rebase since main now has the canonical versions of cancel.ts, sorobanService.ts, and the withdraw test.

@ogazboiz

ogazboiz commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

closing for inactivity — this has been open ~4 weeks with no contributor response after the last review. main has moved significantly since (#499 contract reconciliation, #706 auth-mock fix, ~150 new issues filed). if you want to continue this work, open a fresh PR rebased on current main. happy to review it then.

join us on Telegram if you want to coordinate: https://t.me/+DOylgFv1jyJlNzM0

@ogazboiz ogazboiz closed this Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add code coverage reporting to CI via Codecov

2 participants